Skip to content

refactor(shard): drop journal generic, metrics scrape, poll cursor - #3914

Open
numinnex wants to merge 5 commits into
masterfrom
fix_cleanup_issues_nextnextnext
Open

refactor(shard): drop journal generic, metrics scrape, poll cursor#3914
numinnex wants to merge 5 commits into
masterfrom
fix_cleanup_issues_nextnextnext

Conversation

@numinnex

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the S-waiting-on-review PR is waiting on a reviewer label Aug 18, 2026
@numinnex

Copy link
Copy Markdown
Contributor Author

merge after #3911

@numinnex
numinnex force-pushed the fix_cleanup_issues_nextnextnext branch from 258c662 to d9284e3 Compare August 18, 2026 12:23
@numinnex numinnex changed the title refactor(consensus): single restore constructor + wedge/replay/ack fixes refactor(shard): drop journal generic, metrics scrape, poll cursor Aug 18, 2026
@codecov

codecov Bot commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 53.10651% with 317 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.98%. Comparing base (3cd0860) to head (0e6a624).

Files with missing lines Patch % Lines
core/server/src/partition_reconciler.rs 0.57% 174 Missing ⚠️
core/partitions/src/poll_plan.rs 2.40% 81 Missing ⚠️
core/simulator/src/lib.rs 42.85% 16 Missing ⚠️
core/partitions/src/iggy_index_reader.rs 77.08% 9 Missing and 2 partials ⚠️
core/server/src/http/metrics.rs 45.00% 11 Missing ⚠️
core/server/src/bootstrap.rs 80.95% 7 Missing and 1 partial ⚠️
core/server/src/dispatch.rs 89.58% 5 Missing ⚠️
core/server_common/src/executor.rs 86.11% 5 Missing ⚠️
core/configs/src/server_config/cluster.rs 90.90% 1 Missing ⚠️
core/partitions/src/types.rs 87.50% 1 Missing ⚠️
... and 4 more
Additional details and impacted files
@@              Coverage Diff              @@
##             master    #3914       +/-   ##
=============================================
- Coverage     83.76%   56.98%   -26.79%     
  Complexity     1358     1358               
=============================================
  Files          1212     1209        -3     
  Lines        166189   153732    -12457     
  Branches     133663   121541    -12122     
=============================================
- Hits         139210    87605    -51605     
- Misses        23342    62603    +39261     
+ Partials       3637     3524      -113     
Components Coverage Δ
Rust Core 51.56% <53.10%> (-33.00%) ⬇️
Java SDK 66.67% <ø> (ø)
C# SDK 65.95% <ø> (-10.13%) ⬇️
Python SDK 90.00% <ø> (ø)
PHP SDK 84.48% <ø> (ø)
Node SDK 89.09% <ø> (-6.76%) ⬇️
Go SDK 68.32% <ø> (ø)
Files with missing lines Coverage Δ
core/configs/src/common/server.rs 100.00% <100.00%> (ø)
core/configs/src/server_config/defaults.rs 100.00% <100.00%> (ø)
core/journal/src/lib.rs 25.00% <ø> (ø)
core/journal/src/prepare_journal.rs 41.84% <ø> (-48.61%) ⬇️
core/metadata/src/impls/metadata.rs 70.10% <100.00%> (-16.52%) ⬇️
core/partitions/src/iggy_partition.rs 57.76% <100.00%> (-29.55%) ⬇️
core/partitions/src/journal.rs 70.16% <ø> (-19.24%) ⬇️
core/partitions/src/lib.rs 0.00% <ø> (ø)
core/partitions/src/log.rs 61.84% <100.00%> (-19.08%) ⬇️
core/server/src/auth.rs 80.24% <100.00%> (-9.88%) ⬇️
... and 26 more

... and 523 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@numinnex numinnex changed the title refactor(shard): drop journal generic, metrics scrape, poll cursor refactor(shard): drop journal generic, metrics scrape, poll cursor Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review PR is waiting on a reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant